Skip to content

Conversation

lparsons
Copy link
Member

@lparsons lparsons commented Jan 31, 2025

Description

Checklist:

  • I have previewed changes locally or with CircleCI (runs when PR is created)
  • I have completed any content reviews, such as getting input from relevant working groups. If no, please note this and wait to post the PR to the #website channel until the content has been settled.

When you are ready for a technical review/merge, post the for the link for the PR in the US-RSE Slack (#website) to ask for reviewers.

@lparsons lparsons force-pushed the bhm-2025-randolph branch 2 times, most recently from 6bb2bcb to 78a5900 Compare January 31, 2025 21:05
@lparsons lparsons marked this pull request as ready for review January 31, 2025 21:16
@lparsons
Copy link
Member Author

@uwcdc Here is the PR for your post. I've backdated it to force GitHub to actually render this page, but we can change the date just before we merge if we want.

It would be nice to control the image size a bit more, but my attempts so far have failed. If anyone from @USRSE/usrse-maintainers has any thoughts I'd appreciate it. We can always fix it later if we figure out some way to to do it.

Preview link: https://output.circle-artifacts.com/output/job/6d381af1-3c78-4bb9-aac6-b62950c293b5/artifacts/0/usrse.github.io/2025-01-31-randolph/index.html

Copy link
Contributor

@cdcore09 cdcore09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cmaimone
Copy link
Contributor

To control the size of the images, try using html tags directly instead of markdown image syntax. Or resize the images before uploading them.

@lparsons
Copy link
Member Author

lparsons commented Feb 3, 2025

@USRSE/usrse-maintainers This is ready to merge.

@crd477 crd477 merged commit cfe50fa into USRSE:main Feb 3, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants